-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Localization #6349
Localization #6349
Conversation
Add option of running adaptive localization that can simply be turned on and does not need any user input. Only parameters that are significantly correlated to responses will be updated. Default value of what constitutes significant correlation is calculated based on theory, but can be set by the user.
Codecov Report
@@ Coverage Diff @@
## main #6349 +/- ##
==========================================
+ Coverage 82.64% 83.81% +1.16%
==========================================
Files 347 342 -5
Lines 21390 20810 -580
Branches 862 856 -6
==========================================
- Hits 17678 17442 -236
+ Misses 3413 3073 -340
+ Partials 299 295 -4
... and 16 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Fix typing
Co-authored by: Berent Å. S. Lunde <[email protected]>
@@ -78,6 +78,20 @@ def test_runpath_file(tmpdir, source_root): | |||
assert os.path.isfile("RUNPATH_WORKFLOW_1.OK") | |||
|
|||
|
|||
# def test_reek_case(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to delete this and other test-files before merging.
Pre review checklist
Ground Rules),
and changes to existing code have good test coverage.
Pre merge checklist